Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

update to v7.3.11#99

Merged
mattip merged 14 commits into
conda-forge:mainfrom
mattip:v7.3.10
Jan 18, 2023
Merged

update to v7.3.11#99
mattip merged 14 commits into
conda-forge:mainfrom
mattip:v7.3.10

Conversation

@mattip

@mattip mattip commented Dec 7, 2022

Copy link
Copy Markdown
Contributor

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

Part 1 of #98. Note that in the mean time CPython released security fixes for python3.8.16, python3.9.16, it may be worthwhile adding patches for those.

@conda-forge-linter

Copy link
Copy Markdown

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@mattip

mattip commented Dec 7, 2022

Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please rerender

conda-forge-webservices[bot] and others added 2 commits December 7, 2022 12:37
@mattip

mattip commented Dec 8, 2022

Copy link
Copy Markdown
Contributor Author

CI is passing. Anyone care to review?

@h-vetinari h-vetinari left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI is passing. Anyone care to review?

Not that it says much, but this LGTM :)

@h-vetinari

Copy link
Copy Markdown
Member

Ah, and I was wondering about osx-arm support - it's mentioned in the announcement, but I guess it makes sense to tackle this in a separate PR. Or is cross-compilation still a ways away?

@mattip

mattip commented Dec 10, 2022

Copy link
Copy Markdown
Contributor Author

it makes sense to tackle this in a separate PR

I think so. I don't know if cross-compilation is an option since it will be very slow. I already build aarch64 and ppc64le outside CI, I could do the same for arm64. However I would want buy-in from the conda-forge team, since it would mean starting a migration for binary packages.

@mattip

mattip commented Dec 10, 2022

Copy link
Copy Markdown
Contributor Author
it makes sense to tackle this in a separate PR

xref #88

@h-vetinari

Copy link
Copy Markdown
Member

I don't know if cross-compilation is an option since it will be very slow

Why should it be slow? For aarch/ppc the compilation essentially runs with native (=host) speed. What's slow is emulating one arch through another (e.g. pretend you're aarch64 while running on actual x64 hardware).

But yeah, it would need a new migration (like for pypy-windows at the time), and likely a bunch of effort to get the bootstrap and the first couple of key packages off the ground.

@mattip

mattip commented Dec 10, 2022

Copy link
Copy Markdown
Contributor Author

I think I was confused. cross-compilation (using a different compiler chain and tricking python platform, sys, os, sysconfig and other modules) is not tested and probably will not work out-of-the-box with the PyPy builds. Using qemu on x86 to emulate arm64 will be slow, which is why we build the aarch64 and ppc64le outside of CI.

@mattip

mattip commented Dec 12, 2022

Copy link
Copy Markdown
Contributor Author

This should be patched for issue 3865 so the macos-x86_64 will run on older macos<11. The patch is this commit

@h-vetinari

Copy link
Copy Markdown
Member

@mattip
Could you upgrade to 7.3.11 here?

@mattip mattip mentioned this pull request Jan 8, 2023
5 tasks
@conda-forge-webservices

Copy link
Copy Markdown

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@mattip mattip changed the title update to v7.3.10 update to v7.3.11 Jan 8, 2023
@mattip

mattip commented Jan 8, 2023

Copy link
Copy Markdown
Contributor Author

Could you upgrade to 7.3.11 here?

Done

@mattip

mattip commented Jan 8, 2023

Copy link
Copy Markdown
Contributor Author

This should be patched for issue 3865

Also done, this was the main reason for v7.3.11

@mattip

mattip commented Jan 8, 2023

Copy link
Copy Markdown
Contributor Author

CPython released security fixes for python3.8.16, python3.9.16, it may be worthwhile adding patches for those.

Also done :)

@mattip

mattip commented Jan 9, 2023

Copy link
Copy Markdown
Contributor Author

CI is passing.

Once this is merged I will build the ppc64le and aarch64 artifacts outside of CI according to this gist

@h-vetinari

Copy link
Copy Markdown
Member

I think this is ready, no?

@mattip

mattip commented Jan 16, 2023

Copy link
Copy Markdown
Contributor Author

It has been a week. If no other reviewer does so before, I will merge soon and then create the other artifacts.

@mattip mattip merged commit ef1ddd8 into conda-forge:main Jan 18, 2023
@mattip

mattip commented Jan 20, 2023

Copy link
Copy Markdown
Contributor Author

When trying to follow the procedure to build for aarnch64 and ppc64le I am hitting an error when rerendering.

pkg_resources.extern.packaging.version.InvalidVersion: Invalid version: '3.8.* *_cpython'
complete traceback
$ conda-smithy rerender
INFO:conda_smithy.configure_feedstock:Downloading conda-forge-pinning-2023.01.20.05.27.27
INFO:conda_smithy.configure_feedstock:Extracting conda-forge-pinning to /tmp/tmpvr_7j9ho
INFO:conda_smithy.configure_feedstock:openssl3.yaml from feedstock is ignored and upstream version is used
Setting build platform. This is only useful when pretending to be on another platform, such as for rendering necessary dependencies on a non-native platform. I tru
st that you know what you're doing.
WARNING:conda_build.config:Setting build platform. This is only useful when pretending to be on another platform, such as for rendering necessary dependencies on a
 non-native platform. I trust that you know what you're doing.
Setting build arch. This is only useful when pretending to be on another arch, such as for rendering necessary dependencies on a non-native arch. I trust that you 
know what you're doing.
WARNING:conda_build.config:Setting build arch. This is only useful when pretending to be on another arch, such as for rendering necessary dependencies on a non-nat
ive arch. I trust that you know what you're doing.
No numpy version specified in conda_build_config.yaml.  Falling back to default numpy value of 1.16
WARNING:conda_build.metadata:No numpy version specified in conda_build_config.yaml.  Falling back to default numpy value of 1.16
Adding in variants from internal_defaults
INFO:conda_build.variants:Adding in variants from internal_defaults
Adding in variants from /tmp/tmpvr_7j9ho/conda_build_config.yaml
INFO:conda_build.variants:Adding in variants from /tmp/tmpvr_7j9ho/conda_build_config.yaml
Adding in variants from /home/matti/pypy3.6-feedstock/recipe/conda_build_config.yaml
INFO:conda_build.variants:Adding in variants from /home/matti/pypy3.6-feedstock/recipe/conda_build_config.yaml
INFO:conda_smithy.configure_feedstock:Applying migrations: /tmp/tmpvr_7j9ho/share/conda-forge/migrations/openssl3.yaml
Adding in variants from argument_variants
INFO:conda_build.variants:Adding in variants from argument_variants
INFO:conda_smithy.configure_feedstock:Applying migrations: /tmp/tmpvr_7j9ho/share/conda-forge/migrations/openssl3.yaml
INFO:conda_smithy.configure_feedstock:Applying migrations: /tmp/tmpvr_7j9ho/share/conda-forge/migrations/openssl3.yaml
Traceback (most recent call last):
  File "/home/matti/miniconda3/envs/builder/bin/conda-smithy", line 10, in <module>
    sys.exit(main())
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_smithy/cli.py", line 670, in main
    args.subcommand_func(args)
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_smithy/cli.py", line 486, in __call__
    self._call(args, tmpdir)
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_smithy/cli.py", line 491, in _call
    configure_feedstock.main(
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_smithy/configure_feedstock.py", line 2277, in main
    render_azure(env, config, forge_dir, return_metadata=True)
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_smithy/configure_feedstock.py", line 1389, in render_azure
    return _render_ci_provider(
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_smithy/configure_feedstock.py", line 690, in _render_ci_provider
    metas = conda_build.api.render(
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/api.py", line 38, in render
    metadata_tuples = render_recipe(recipe_path, bypass_env_check=bypass_env_check,
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/render.py", line 849, in render_recipe
    rendered_metadata = distribute_variants(m, variants,
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/render.py", line 711, in distribute_variants
    used_variables = metadata.get_used_loop_vars(force_global=False)
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/metadata.py", line 2558, in get_used_loop_vars
    for var in self.get_used_vars(
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/metadata.py", line 2661, in get_used_vars
    meta_yaml_reqs = self._get_used_vars_meta_yaml(
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/metadata.py", line 2746, in _get_used_vars_meta_yaml
    reqs_text, recipe_text = self._get_used_vars_meta_yaml_helper(
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/metadata.py", line 2714, in _get_used_vars_meta_yaml_helper
    recipe_text = self.get_recipe_text(
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/metadata.py", line 2013, in get_recipe_text
    ns_cfg(self.config),
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/metadata.py", line 102, in ns_cfg
    defaults = variants.get_default_variant(config)
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/variants.py", line 109, in get_default_variant
    base.update(_get_default_compilers(config.platform, python))
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/conda_build/variants.py", line 90, in _get_default_compilers
    if parse_version(py_ver) >= parse_version('3.5'):
  File "/home/matti/miniconda3/envs/builder/lib/python3.9/site-packages/pkg_resources/_vendor/packaging/version.py", line 266, in __init__
    raise InvalidVersion(f"Invalid version: '{version}'")
pkg_resources.extern.packaging.version.InvalidVersion: Invalid version: '3.8.* *_cpython'
``

@mattip

mattip commented Jan 20, 2023

Copy link
Copy Markdown
Contributor Author

The error above is on aarch64. On ppc64le that error is a warning.

@mattip

mattip commented Jan 22, 2023

Copy link
Copy Markdown
Contributor Author

Starting over from an updated base environment seems to have allowed the build process to complete.

@mattip

mattip commented Jan 25, 2023

Copy link
Copy Markdown
Contributor Author

@isuruf the conda-forge-p8-1 machine is out of disk space. Is there a way to clean up something there or to get a larger disk?

@mattip

mattip commented Jan 26, 2023

Copy link
Copy Markdown
Contributor Author

@beckermr do you known anything about the ppc64le conda-forge-p8-1 machine? It does not have enough disk space to allow me to build the ppc64le packages for this feedstock, as I have been able to do in the past following this gist. The builds are too slow to use a cross-compilation workflow.

@isuruf

isuruf commented Jan 26, 2023

Copy link
Copy Markdown
Member

Cleaned up the VM. Have a look now.

@mattip

mattip commented Jan 31, 2023

Copy link
Copy Markdown
Contributor Author

I uploaded ppc64le and aarch64 tarballs to https://anaconda.org/mattip/pypy3.8/files and https://anaconda.org/mattip/pypy3.9/files and marked them with the "copy" label. There should be 4 files in each location.

Here are the build logs

ppc64le-logs.txt
aarch64-logs.txt

xref @isuruf could you copy the files to https://anaconda.org/conda-forge/pypy3.8 and https://anaconda.org/conda-forge/pypy3.9 ?

@isuruf

isuruf commented Feb 1, 2023

Copy link
Copy Markdown
Member

Done

@mattip

mattip commented Feb 1, 2023

Copy link
Copy Markdown
Contributor Author

Thanks!

@h-vetinari h-vetinari mentioned this pull request Aug 29, 2023
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants